Skip to content

Feature/leaderboard command#1465

Open
barsh404error wants to merge 1 commit into
Together-Java:developfrom
barsh404error:feature/leaderboard-command
Open

Feature/leaderboard command#1465
barsh404error wants to merge 1 commit into
Together-Java:developfrom
barsh404error:feature/leaderboard-command

Conversation

@barsh404error
Copy link
Copy Markdown
Member

@barsh404error barsh404error commented Apr 25, 2026

What

Adds a /leaderboard command that displays the all-time top helpers. It reads the hall-of-fame channel history, finds messages that mention top helpers, counts how many times each user was awarded, and shows the top 10 ranked with medals.

How

Reads the hall-of-fame channel history, filters messages containing "top helper", extracts mentioned users via getMentions(), counts wins per user and shows the top 10 in an embed.

Screenshot From 2026-05-26 21-56-39

Edge cases

  • Channel not found: yes, returns "Could not find the hall of fame channel."
  • No messages in channel: yes, sorted.isEmpty() catches it, then returns "No top helper data found."
  • Messages exist but none contain "top helper": yes, the same sorted.isEmpty() check covers it.
  • Multiple users in one message: yes, the getMentions().getUsers() loop handles multiple mentions per message.
Screenshot From 2026-05-27 15-32-30 Screenshot From 2026-05-27 15-32-54 Screenshot From 2026-05-27 15-42-14

NOTE: Channel not found and Messages exist but none contain "top helper" returns the same output "No top helper data found."

@barsh404error barsh404error requested a review from a team as a code owner April 25, 2026 09:18
@barsh404error barsh404error force-pushed the feature/leaderboard-command branch from 8a9840f to ff6632d Compare April 25, 2026 09:23
@barsh404error barsh404error self-assigned this Apr 25, 2026
@barsh404error barsh404error force-pushed the feature/leaderboard-command branch from ff6632d to 9bb9883 Compare April 25, 2026 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant